ref: Simplify proxy handler and remove caching layer#4
Conversation
Replace catch-all route with explicit route handlers for each endpoint. Remove in-memory cache and stale-while-error logic in favor of direct proxy pass-through. Add typed Context and ContentfulStatusCode imports. Co-Authored-By: Claude <noreply@anthropic.com> Agent transcript: https://claudescope.sentry.dev/share/4rxSgfqf_8e03QyKF2z9P75vink8hZ2eN_MRVyURJa4
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| if (path.startsWith('/skills/')) { | ||
| return c.redirect(path.slice('/skills'.length), 301); | ||
| } | ||
|
|
There was a problem hiding this comment.
Missing route for /SKILL_TREE.md endpoint returns 404
High Severity
The /SKILL_TREE.md route that was previously handled by mapPath (which mapped both / and /SKILL_TREE.md to ${BASE}/SKILL_TREE.md) is now missing. The /:skill/* route won't match it because Hono's /:param/* pattern requires at least two path segments, and /SKILL_TREE.md is a single segment. Requests to /SKILL_TREE.md will now return 404 instead of the expected skill tree content. The PR test plan explicitly lists this route as needing verification.
1 participant
Summary
ContextandContentfulStatusCodeimports for better type safetyTest plan
/,/SKILL_TREE.md,/sdks,/workflows,/featuresroutes return correct content/:skill/SKILL.mdand/:skill/*wildcard routes work🤖 Generated with Claude Code
Agent transcript: https://claudescope.sentry.dev/share/5b7dB5WQJ6ikZ7Tq0RW_lLQ4mjPrfZ3qmESX_KU8Sog